Skip to content
This repository has been archived by the owner on Jun 7, 2021. It is now read-only.

TRAFODION-2995 : modifying project and foundation links. #1480

Open
wants to merge 4 commits into
base: master
Choose a base branch
from
Open

TRAFODION-2995 : modifying project and foundation links. #1480

wants to merge 4 commits into from

Conversation

PierreSmits
Copy link
Member

first batch of corrections.
also included correction to download location of latest release (dyn/closer ref).

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2490/

@Traf-Jenkins
Copy link

@PierreSmits
Copy link
Member Author

I can't see where the tests failed.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2491/

@Traf-Jenkins
Copy link

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2492/

@Traf-Jenkins
Copy link

@svarnau
Copy link
Contributor

svarnau commented Mar 18, 2018

https://jenkins.esgyn.com/job/static-checks/2849/consoleFull

2018-03-17 12:27:41 Unapproved licenses:
2018-03-17 12:27:41
2018-03-17 12:27:41 trafodion/Makefile
2018-03-17 12:27:41 trafodion/doap.rdf
2018-03-17 12:27:41 trafodion/env.sh
2018-03-17 12:27:41 trafodion/pom.xml
2018-03-17 12:27:41 trafodion/core/sql/common/11_ut2l.h
2018-03-17 12:27:41 trafodion/core/sql/common/1n_lt2u.h
2018-03-17 12:27:41 trafodion/core/sql/eh/00-readme
2018-03-17 12:27:41 trafodion/core/sql/export/AbortCallBack.h
2018-03-17 12:27:41 trafodion/docs/lob_guide/src/asciidoc/_chapters/about.adoc
2018-03-17 12:27:41 trafodion/docs/messages_guide/src/asciidoc/_chapters/about.adoc
2018-03-17 12:27:41 trafodion/docs/odb_user/src/asciidoc/_chapters/a_warnings.adoc

RAT apparently does not like https in the license statements.

@PierreSmits
Copy link
Member Author

Hmm.

Is that something we can fix in our RAT configurations?

@svarnau
Copy link
Contributor

svarnau commented Mar 19, 2018

I thought maybe it would be addressed in a later version of RAT than what we use, but I see no indication of that in their docs.
Maybe we have to stick with exact license text, even for the old http URL.

@Traf-Jenkins
Copy link

New Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2494/

@Traf-Jenkins
Copy link

@DaveBirdsall
Copy link
Contributor

jenkins, retest

@Traf-Jenkins
Copy link

Check Test Started: https://jenkins.esgyn.com/job/Check-PR-master/2548/

@Traf-Jenkins
Copy link

@DaveBirdsall
Copy link
Contributor

Just checking up on this pull request to see if we can move it along. It looks like the only issue remaining is to change "https" back to "http" in the Apache license headers. @svarnau and @PierreSmits, do you agree?

@PierreSmits
Copy link
Member Author

It seems to me that having either https or http in the licence header should not invalidate the test.

@svarnau
Copy link
Contributor

svarnau commented Apr 11, 2018

We are just going by what RAT reports. There has been some discussion of this on the legel-discuss mail list, and it was suggested that even changing http to https in the license text is a change that would need to be dicussed and approved by the ASF.

Sign up for free to subscribe to this conversation on GitHub. Already have an account? Sign in.
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

4 participants